Skip to content

Conversation

@saifsultanc
Copy link
Contributor

Context

⛑️ Ticket(s): https://secure.helpscout.net/conversation/2857547361/78511

Summary

To use the default form confirmations in Entry Blocks.

https://www.loom.com/share/4bdd738bbd0342548807c2dffd93bd73

…display default confirmations when editing entries via Entry Blocks.
@github-actions
Copy link

Warnings
⚠️ When ready, don't forget to request reviews on this pull request from your fellow wizards.

Generated by 🚫 dangerJS against e41cc1b

Copy link
Contributor

@veryspry veryspry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Simple and clean ☺️

* @return string
*/
add_filter( 'gpeb_edit_confirmation', function ( $confirmation, $form, $entry ) {

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Non-blocker: I thiiiinkkk it's standard in the snippet library to use variables for "scoped values" such as form ids.

Suggested change
// Replace with your form ID.
$form_id = 123;

@saifsultanc saifsultanc merged commit d3702e1 into master Mar 4, 2025
2 of 3 checks passed
@saifsultanc saifsultanc deleted the saif/add/78511-snippet-default-confirmation branch March 4, 2025 03:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants